程序员练级攻略：程序员修养-[2026重制版]

## 问题描述
一句话概括你遇到的问题
 
## 环境
- 操作系统: macOS 15.2 / Ubuntu 22.04 / Windows 11
- JDK版本: OpenJDK 21.0.2
- Spring Boot: 3.4.1
- 其他相关依赖及版本:
 
## 期望行为
描述你认为应该发生什么
 
## 实际行为
描述实际发生了什么（附截图/日志）
 
## 复现步骤
1. 第一步做什么
2. 第二步做什么
3. ...
 
## 最小代码示例
\`\`\`java
// 这里贴上能复现问题的最小代码
\`\`\`
 
## 错误日志
\`\`\`
Exception in thread "main" ...
\`\`\`
 
## 已尝试的解决方案
- [ ] 方案1: 结果...
- [ ] 方案2: 结果...
 
## 参考资料
- 相关文档链接:
- 类似问题链接:

// ❌ 差：变量命名不清晰，逻辑嵌套深
public List<User> getData(List<String> ids, String type, boolean flag) {
    List<User> res = new ArrayList<>();
    if (ids != null) {
        for (String id : ids) {
            if (type.equals("A")) {
                if (flag) {
                    User u = repo.findById(id);
                    if (u != null && u.isActive()) {
                        res.add(u);
                    }
                }
            }
        }
    }
    return res;
}
 
// ✅ 好：清晰的命名，早返回，逻辑清晰
public List<User> findActiveUsersByIds(
        List<String> userIds,
        UserType userType,
        boolean includeInactive) {
 
    if (CollectionUtils.isEmpty(userIds)) {
        return Collections.emptyList();
    }
 
    return userIds.stream()
        .map(userRepository::findById)
        .filter(Objects::nonNull)
        .filter(user -> matchesCriteria(user, userType, includeInactive))
        .toList();
}
 
private boolean matchesCriteria(User user, UserType type, boolean includeInactive) {
    boolean typeMatches = type == UserType.ALL || user.getType() == type;
    boolean activeMatches = includeInactive || user.isActive();
    return typeMatches && activeMatches;
}

@ExtendWith(MockitoExtension.class)
@DisplayName("用户服务单元测试")
class UserServiceTest {
 
    @Mock
    private UserRepository userRepository;
 
    @Mock
    private PasswordEncoder passwordEncoder;
 
    @InjectMocks
    private UserService userService;
 
    // ========== 测试注册场景 ==========
 
    @Nested
    @DisplayName("注册功能")
    class RegisterTests {
 
        @Test
        @DisplayName("应该成功注册新用户")
        void shouldRegisterNewUserSuccessfully() {
            // Given - 准备测试数据
            RegisterRequest request = new RegisterRequest(
                "testuser",
                "SecurePass123!",
                "test@example.com"
            );
            when(passwordEncoder.encode(anyString())).thenReturn("encoded_hash");
            when(userRepository.existsByUsername("testuser")).thenReturn(false);
            when(userRepository.save(any())).thenAnswer(invocation -> invocation.getArgument(0));
 
            // When - 执行操作
            User result = userService.register(request);
 
            // Then - 验证结果
            assertThat(result.getUsername()).isEqualTo("testuser");
            assertThat(result.getPasswordHash()).isEqualTo("encoded_hash");
            assertThat(result.getEmail()).isEqualTo("test@example.com");
            verify(passwordEncoder).encode("SecurePass123!");
            verify(userRepository).save(any(User.class));
        }
 
        @Test
        @DisplayName("当用户名已存在时应抛出异常")
        void shouldThrowWhenUsernameExists() {
            RegisterRequest request = new RegisterRequest(
                "existinguser", "pass", "email@test.com"
            );
            when(userRepository.existsByUsername("existinguser")).thenReturn(true);
 
            assertThatThrownBy(() -> userService.register(request))
                .isInstanceOf(DuplicateUsernameException.class)
                .hasMessageContaining("用户名已存在");
 
            verify(userRepository, never()).save(any());
        }
 
        @Test
        @DisplayName("密码强度不足时应抛出异常")
        void shouldThrowWhenPasswordWeak() {
            RegisterRequest request = new RegisterRequest("user", "123", "email@test.com");
 
            assertThatThrownBy(() -> userService.register(request))
                .isInstanceOf(WeakPasswordException.class);
        }
    }
}

// ❌ 过于信任调用者
public void processUserInput(String input) {
    // 直接使用，假设input不为null且格式正确
    int value = Integer.parseInt(input);
    database.save(value);
}
 
// ✅ 防御性编程
public void processUserInput(String input) {
    // 1. 参数校验
    if (input == null || input.isBlank()) {
        throw new IllegalArgumentException("输入不能为空");
    }
 
    // 2. 格式验证
    if (!input.matches("\\d+")) {
        throw new IllegalArgumentException("输入必须是数字");
    }
 
    // 3. 边界检查
    int value = Integer.parseInt(input);
    if (value < 0 || value > MAX_ALLOWED_VALUE) {
        throw new IllegalArgumentException("数值超出允许范围");
    }
 
    // 4. 异常捕获
    try {
        database.save(value);
    } catch (DataAccessException e) {
        log.error("保存数据失败, value={}", value, e);
        throw new BusinessException("数据处理失败，请稍后重试");
    }
}

# 上线前检查清单
 
## 功能测试
- [ ] 所有需求都已实现并通过验收
- [ ] 回归测试通过，无已知严重Bug
- [ ] 边界情况和异常流程已测试
 
## 性能测试
- [ ] 压力测试完成，明确系统容量上限
- [ ] P99响应时间满足SLA要求
- [ ] 数据库慢查询已优化
 
## 安全检查
- [ ] OWASP Top 10漏洞扫描通过
- [ ] 依赖包无已知CVE漏洞
- [ ] 敏感信息已从代码中移除
- [ ] HTTPS证书有效且配置正确
- [ ] 安全头（CSP, X-Frame-Options等）已配置
 
## 监控告警
- [ ] 关键业务指标监控已接入
- [ ] 错误日志收集和告警已配置
- [ ] 告警通知渠道畅通（钉钉/企微/邮件）
- [ ] 链路追踪（APM）已启用
 
## 运维准备
- [ ] 部署脚本和文档完整
- [ ] 回滚方案已准备并可执行
- [ ] 数据库迁移脚本已审查
- [ ] 配置管理（环境变量/配置中心）就绪
 
## 文档
- [ ] API文档更新（Swagger/OpenAPI）
- [ ] 运维手册更新
- [ ] 变更记录完整